Skip to content

[Reporting] Suppress debug logs in the mock logger#116012

Merged
tsullivan merged 3 commits intoelastic:masterfrom
tsullivan:reporting/mock-logger-quieter
Oct 26, 2021
Merged

[Reporting] Suppress debug logs in the mock logger#116012
tsullivan merged 3 commits intoelastic:masterfrom
tsullivan:reporting/mock-logger-quieter

Conversation

@tsullivan
Copy link
Copy Markdown
Member

@tsullivan tsullivan commented Oct 21, 2021

Summary

When running Reporting Jest tests, there were too many debug logs shown in the console. This PR comments out the line of code that causes debug logs in the console, so that a developer can locally uncomment when they need to troubleshoot a failing test.

Follows: #115881 (comment)

@tsullivan tsullivan requested review from a team as code owners October 21, 2021 20:10
@tsullivan tsullivan added release_note:skip Skip the PR/issue when compiling release notes Team:AppServicesUx v8.0.0 labels Oct 21, 2021
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-reporting-services (Team:Reporting Services)

@tylersmalley
Copy link
Copy Markdown
Member

@elasticmachine merge upstream

@tsullivan
Copy link
Copy Markdown
Member Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Test Failures

  • [job] [logs] Default CI Group #10 / Uptime app with real-world data feature controls security no uptime privileges "before all" hook for "doesn't show uptime navlink"

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@tsullivan tsullivan added the backport:skip This PR does not require backporting label Oct 26, 2021
@tsullivan tsullivan merged commit 3b8766f into elastic:master Oct 26, 2021
@tsullivan tsullivan deleted the reporting/mock-logger-quieter branch October 26, 2021 20:29
@tylersmalley tylersmalley added v7.16.0 auto-backport Deprecated - use backport:version if exact versions are needed and removed backport:skip This PR does not require backporting labels Oct 26, 2021
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Oct 26, 2021
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Backport successful

Status Branch Result
7.16

This backport PR will be merged automatically after passing CI.

jloleysens added a commit to jloleysens/kibana that referenced this pull request Oct 27, 2021
…-migrate-away-from-injected-css-js

* 'master' of github.com:elastic/kibana: (61 commits)
  [ML] Nodes overview for the Model Management page (elastic#116361)
  [Uptime] Uptime index config using kibana.yml (elastic#115775)
  [Controls] Dashboard Integration (elastic#115991)
  skip flaky suite (elastic#104260)
  Include Files in GitHub UI (elastic#115956)
  skip flaky suite (elastic#116060)
  [Canvas] By-Value Embeddables (elastic#113827)
  Skip failing test (elastic#115366)
  [Osquery] Fix live query search doesn't return relevant results for agents (elastic#116332)
  [Integrations] Added link in old Add Data description and fixed alignment in cards (elastic#116213)
  [Actions] Extended ActionTypeRegistry with connector validation to validate config with secrets (elastic#116079)
  skip flaky suite (elastic#109329)
  Grant access to machine learning features when base privileges are used (elastic#115444)
  Skipping failing test (elastic#84957)
  [RAC][Security Solution] Adds migration to new SecuritySolution rule types (elastic#112113)
  skip flaky suite (elastic#115366)
  [Fleet] Marking API spec as experimental (elastic#116331)
  [Docs] Cleaning up the versions in the upgrade paths. Closes elastic#116223 (elastic#116228)
  [Reporting] Suppress debug logs in the mock logger (elastic#116012)
  [Metrics UI] Clear threshold alert groups state when filterQuery changes (elastic#116205)
  ...

# Conflicts:
#	src/plugins/dashboard/public/application/embeddable/dashboard_container.tsx
#	src/plugins/dashboard/public/types.ts
kibanamachine added a commit that referenced this pull request Oct 27, 2021
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Tim Sullivan <tsullivan@users.noreply.github.com>
@sophiec20 sophiec20 added the zDeprecated Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead label Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes v7.16.0 v8.0.0 zDeprecated Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants